Skip to content

Replaces ProtocolClassValidator for the second example #11006

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

krisdante
Copy link
Contributor

Examples for Class Constraint Validator use ContainsAlphanumeric that was added as property validator. It will not work for a class.
The intention of the second example seems to be the usage of ProtocolClassValidator introduced in this section (chapter).

Examples for Class Constraint Validator use ContainsAlphanumeric that was added as property validator. It will not work for a class. 
The intention of the second example seems to be the usage of ProtocolClassValidator introduced in this section (chapter).
@javiereguiluz javiereguiluz added this to the 3.4 milestone Feb 18, 2019
@javiereguiluz
Copy link
Member

@krisdante good catch! Thanks for fixing this and congrats on your first Symfony Docs contribution.

javiereguiluz added a commit that referenced this pull request Feb 18, 2019
…krisdante)

This PR was submitted for the 4.2 branch but it was merged into the 3.4 branch instead (closes #11006).

Discussion
----------

Replaces ProtocolClassValidator for the second example

Examples for Class Constraint Validator use ContainsAlphanumeric that was added as property validator. It will not work for a class.
The intention of the second example seems to be the usage of ProtocolClassValidator introduced in this section (chapter).

<!--

If your pull request fixes a BUG, use the oldest maintained branch that contains
the bug (see https://symfony.com/roadmap for the list of maintained branches).

If your pull request documents a NEW FEATURE, use the same Symfony branch where
the feature was introduced (and `master` for features of unreleased versions).

-->

Commits
-------

42007cb Replaces ProtocolClassValidator for the second example
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants